Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 14 day cooldown withdraw, deposit min #16

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

sambacha
Copy link
Contributor

@sambacha sambacha commented Aug 9, 2024

THIS CHECKLIST IS NOT COMPLETE. Use --show-ignored-findings to show all the results.
Summary

unchecked-transfer

Impact: High
Confidence: Medium

src/FOLD_StakingV2.sol#L366-L400

src/FOLD_StakingV2.sol#L267-L282

src/FOLD_StakingV2.sol#L193-L231

src/FoldCaptiveStaking.sol#L358-L392

src/FoldCaptiveStaking.sol#L358-L392

src/FOLD_StakingV2.sol#L193-L231

src/FoldCaptiveStaking.sol#L290-L327

src/FoldCaptiveStaking.sol#L185-L223

src/FoldCaptiveStaking.sol#L277-L286

src/FOLD_StakingV2.sol#L193-L231

src/FOLD_StakingV2.sol#L234-L264

src/FoldCaptiveStaking.sol#L226-L256

src/FoldCaptiveStaking.sol#L259-L274

src/FoldCaptiveStaking.sol#L290-L327

src/FOLD_StakingV2.sol#L267-L282

src/FOLD_StakingV2.sol#L298-L335

src/FOLD_StakingV2.sol#L366-L400

src/FoldCaptiveStaking.sol#L185-L223

src/FoldCaptiveStaking.sol#L185-L223

src/FOLD_StakingV2.sol#L298-L335

src/FoldCaptiveStaking.sol#L226-L256

src/FOLD_StakingV2.sol#L234-L264

src/FoldCaptiveStaking.sol#L259-L274

src/FOLD_StakingV2.sol#L193-L231

src/FOLD_StakingV2.sol#L285-L294

src/FoldCaptiveStaking.sol#L185-L223

divide-before-multiply

Impact: Medium
Confidence: Medium

src/libraries/TickMath.sol#L32-L36

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L39-L43

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

src/libraries/TickMath.sol#L50-L109

incorrect-equality

Impact: Medium
Confidence: High

src/FoldCaptiveStaking.sol#L69-L103

src/FOLD_StakingV2.sol#L77-L111

reentrancy-no-eth

Impact: Medium
Confidence: Medium

src/FoldCaptiveStaking.sol#L290-L327

src/FoldCaptiveStaking.sol#L226-L256

src/FOLD_StakingV2.sol#L77-L111

src/FoldCaptiveStaking.sol#L185-L223

src/FOLD_StakingV2.sol#L285-L294

src/FOLD_StakingV2.sol#L298-L335

src/FoldCaptiveStaking.sol#L185-L223

src/FOLD_StakingV2.sol#L193-L231

src/FoldCaptiveStaking.sol#L69-L103

src/FoldCaptiveStaking.sol#L277-L286

src/FOLD_StakingV2.sol#L366-L400

src/FOLD_StakingV2.sol#L267-L282

src/FoldCaptiveStaking.sol#L259-L274

src/FOLD_StakingV2.sol#L234-L264

src/FOLD_StakingV2.sol#L193-L231

src/FoldCaptiveStaking.sol#L358-L392

unused-return

Impact: Medium
Confidence: Medium

src/FOLD_StakingV2.sol#L77-L111

src/FOLD_StakingV2.sol#L77-L111

src/FOLD_StakingV2.sol#L77-L111

src/FoldCaptiveStaking.sol#L69-L103

src/FoldCaptiveStaking.sol#L69-L103

src/FoldCaptiveStaking.sol#L69-L103

pess-dubious-typecast

Impact: Medium
Confidence: High

src/FoldCaptiveStaking.sol#L358-L392

src/FOLD_StakingV2.sol#L366-L400

src/libraries/TickMath.sol#L116-L271

src/FOLD_StakingV2.sol#L285-L294

src/FoldCaptiveStaking.sol#L226-L256

src/FOLD_StakingV2.sol#L234-L264

src/FOLD_StakingV2.sol#L298-L335

src/libraries/TickMath.sol#L50-L109

src/FoldCaptiveStaking.sol#L290-L327

src/FoldCaptiveStaking.sol#L259-L274

src/FoldCaptiveStaking.sol#L277-L286

src/FOLD_StakingV2.sol#L267-L282

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants